-
-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: implement mutate command #199
base: main
Are you sure you want to change the base?
Conversation
@vbatts This is a pretty gross version of implementing the update command. Looking for feedback. I wish there was an API that would allow me to mutate the tree and understand the semantics of a mtree more. |
@vbatts I'm interested in getting this released so Bazel users have a |
understood. I'm trying to understand if the output of the |
And I'm brainstorming about your comment on how to help understand the semantics of the mtree better. |
Ping! @thesayyn I think it's your turn here :) |
Sorry this slipped away. |
I think @thesayyn is too underwater to get back to this at any point. Also the maintainer of bazel's rules_pkg found a new spot at Google so it's not going away. That means the urgency here got lower. I've tried to sell @rickvanprim on the idea of picking up the PR, but I know it's an obscure hobby project. Maybe we should just close this and open an issue or discussion to suggest the feature without "licking the cookie" of attempting an implementation. |
ah sorry. things have been so busy on my side as well, else I would've taken this over |
Sahin suggested that maybe on the Bazel side we can just hook up a fixture to run hermetic awk interpreter, then write the mtree mutations in awk scripts |
doing things with awk sounds involved... |
If you imagine an engineer is equally comfortable coding in AWK and Go, then I think the former is the more expressive language that's purpose-built for basic manipulation of lines in a file. The problem for us is simply that we want to be hermetic in Bazel, and don't already have a toolchain to get a specific release of awk we can rely on everywhere. |
No description provided.